Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #78 Add new option toggle_list #94

Merged
merged 2 commits into from
Oct 13, 2022

Conversation

nao-pon
Copy link
Collaborator

@nao-pon nao-pon commented Sep 19, 2022

The toggled property is a setting that does not retain the On state. Set this if your AC properties are toggle function.

toggle_list: #optional - default []
- Beep
- Clean
- Econo
- Filter
- Light
- Quiet
- Sleep
- SwingH
- SwingV
- Turbo

The toggled property is a setting that does not retain the On state. Set
this if your AC properties are toggle function.

toggle_list: #optional - default []
    - Beep
    - Clean
    - Econo
    - Filter
    - Light
    - Quiet
    - Sleep
    - SwingH
    - SwingV
    - Turbo
@nagyrobi
Copy link
Contributor

nagyrobi commented Oct 11, 2022

@nao-pon How does this toggle list work? What is it for?

@nao-pon
Copy link
Collaborator Author

nao-pon commented Oct 12, 2022

This is a fix for the toggle command reported in #78.

Toggle commands switch On/Off by sending On. If this integration remembers the On state, it will always send On when sending other commands, which is not the correct behavior.

This PR does not save the On value for the specified command and only sends the On value by service call.

However, since I don't have the corresponding device, I can't verify the operation, and I'm still waiting for a reply at #78.

@nao-pon nao-pon merged commit 163c9d1 into hristo-atanasov:master Oct 13, 2022
@nao-pon nao-pon deleted the toggle_list branch October 13, 2022 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants